Skip to content

tutorial_gmail_automation#544

Open
maithili74 wants to merge 1 commit intomasterfrom
tutorial_gmail_automation
Open

tutorial_gmail_automation#544
maithili74 wants to merge 1 commit intomasterfrom
tutorial_gmail_automation

Conversation

@maithili74
Copy link

@maithili74 maithili74 commented May 26, 2025

Request to review the gmail automation task

Issue: #413

Pre-commit checks:
All checks passed ✅
@maithili74 maithili74 self-assigned this May 26, 2025
@maithili74 maithili74 added the PR_for_reviewers The PR needs to be reviewed by RPs label May 26, 2025
@maithili74 maithili74 requested a review from gpsaggese May 26, 2025 18:00
@maithili74 maithili74 marked this pull request as ready for review May 26, 2025 18:11
@gpsaggese gpsaggese requested a review from Shaunak01 June 2, 2025 12:03
@gpsaggese
Copy link
Collaborator

@Shaunak01 can you pls do a first round of review?

@@ -0,0 +1,939 @@
{
Copy link
Contributor

@Shaunak01 Shaunak01 Jun 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you pls pair with a .py file.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is documentation on how to use jupytext in the docs

Also can you start moving the code in a Python utils file so the notebook contains only the invocations?
You can search in the doc how we write notebooks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR_for_reviewers The PR needs to be reviewed by RPs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants